Skip to content

doc: we got coercion going on here, so no need to be this explicit #36314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 10, 2016
Merged

doc: we got coercion going on here, so no need to be this explicit #36314

merged 1 commit into from
Sep 10, 2016

Conversation

tshepang
Copy link
Member

@tshepang tshepang commented Sep 7, 2016

No description provided.

@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Sep 7, 2016

📌 Commit b99c5cf has been approved by GuillaumeGomez

@Manishearth Manishearth mentioned this pull request Sep 10, 2016
Closed
Manishearth added a commit to Manishearth/rust that referenced this pull request Sep 10, 2016
@Manishearth
Rollup merge of rust-lang#36314 - tshepang:not-needed, r=GuillaumeGomez
37524de 
doc: we got coercion going on here, so no need to be this explicit
@GuillaumeGomez GuillaumeGomez mentioned this pull request Sep 10, 2016
Merged
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Sep 10, 2016
@GuillaumeGomez
Rollup merge of rust-lang#36314 - tshepang:not-needed, r=GuillaumeGomez
a221ad0 
doc: we got coercion going on here, so no need to be this explicit
bors added a commit that referenced this pull request Sep 10, 2016
@bors
Auto merge of #36378 - GuillaumeGomez:rollup, r=GuillaumeGomez
f8ba7cb 
Rollup of 6 pull requests

- Successful merges: #35691, #36045, #36311, #36314, #36326, #36346
- Failed merges:
@bors
Copy link
Collaborator

bors commented Sep 10, 2016

☔ The latest upstream changes (presumably #36378) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors merged commit b99c5cf into rust-lang:master Sep 10, 2016
@tshepang tshepang deleted the not-needed branch September 11, 2016 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aturon aturon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tshepang @rust-highfive @GuillaumeGomez @bors @aturon